Skip to content

fix(app:linux): prevent infinite loop on file backstage#15362

Open
Quwex wants to merge 2 commits intomainfrom
private/quwex/fix-15291
Open

fix(app:linux): prevent infinite loop on file backstage#15362
Quwex wants to merge 2 commits intomainfrom
private/quwex/fix-15291

Conversation

@Quwex
Copy link
Copy Markdown
Contributor

@Quwex Quwex commented Apr 2, 2026

Checklist

  • I have run make prettier-write and formatted the code.
  • All commits have Change-Id
  • I have run tests with make check
  • I have issued make run and manually verified that everything looks okay
  • Documentation (manuals or wiki) has been updated or is not required

@mmeeks
Copy link
Copy Markdown
Contributor

mmeeks commented Apr 8, 2026

Would love to get it fixed; can you re-base ? :-)

Quwex added 2 commits April 9, 2026 08:33
…tainer

refreshVisibleAreaRectangle() enters an infinite retry loop when the
document container is hidden and the anchor section has zero size.
BackstageView in CODA triggers this by hiding #document-container
while multi-page view is active.

Return early when the anchor has zero dimensions.

Includes a coda-qt WebdriverIO regression test.

cool#15291

Signed-off-by: Sarper Akdemir <sarper.akdemir@collabora.com>
Change-Id: I39d99c9d32d92a8892634cfe42f4b078b9e149a3
Move duplicated file-open-via-native-dialog logic into a reusable
openFixture() helper in qt/test/lib/file-dialog.ts.

Signed-off-by: Sarper Akdemir <sarper.akdemir@collabora.com>
Change-Id: I1a1d9177dd82ea50be099844372c7d61d6d74461
@Quwex Quwex force-pushed the private/quwex/fix-15291 branch from 00d46ff to 04e4362 Compare April 9, 2026 06:34
@Quwex Quwex requested a review from gokaysatir April 9, 2026 06:35
@Quwex
Copy link
Copy Markdown
Contributor Author

Quwex commented Apr 9, 2026

@gokaysatir I believe ViewLayoutMultipage is an area you were working on, if so I'd appreciate it if you could review the change :)

Copy link
Copy Markdown
Contributor

@gokaysatir gokaysatir left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Of course. The layout code change looks good to me.

@github-project-automation github-project-automation bot moved this from To Review to To Test in Collabora Online Apr 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: To Test

Development

Successfully merging this pull request may close these issues.

Linux Desktop becomes unresponsive when switching to file tab in multi-page view

3 participants